Propagate deserialization errors with data upstream to facilitate easier use of not-fully compliant third party LLMs #423

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

ple1n wants to merge 2 commits into 64bit:main from ple1n:main

ple1n commented Aug 23, 2025

I used ByteString which makes sure it displays json in string, when you enable tracing logs.
Deserialization errors are loaded with data, so upstream can handle them serving as protocol extension to OpenAI API.

ple1n added 2 commits

August 24, 2025 06:27


          propagate deser errors upstream

843020d


          add serde for openai config. use vendored thiserror

8372bca

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet